Skip to content

Conversation

drichards-87
Copy link
Contributor

What does this PR do?

Removes Docs as CODEOWNERS from the repo. As part of a project to reduce courtesy reviews, the Docs Team is limiting CODEOWNERS entries to repos that single-source content into the docs site or where our review is critical. We’re still happy to review content if you reach out to us directly.

Additional Notes

Review checklist

Please check relevant items below:

  • This PR includes all newly recorded cassettes for any modified tests.

  • This PR does not rely on API client schema changes.

    • The CI should be fully passing.
  • Or, this PR relies on API schema changes and this is a Draft PR to include tests for that new functionality.

    • Note: CI shouldn't be run on this Draft PR, as its expected to fail without the corresponding schema changes.

Removes Docs as CODEOWNERS from the repo. As part of a project to reduce courtesy reviews, the Docs Team is limiting CODEOWNERS entries to repos that single-source content into the docs site or where our review is critical. We’re still happy to review content if you reach out to us directly.
@drichards-87
Copy link
Contributor Author

@NaOHman Thanks for approving this PR. I tried adding it to the merge queue, but it failed since it still needs approval from the Web Frameworks team (your approval shows up under the Web Frameworks Approvers group). Would you be able to merge it directly for me?

@skarimo skarimo merged commit 1c00fa8 into master Sep 26, 2025
12 checks passed
@skarimo skarimo deleted the drichards-87/remove-docs-team-codeowners branch September 26, 2025 17:02
github-actions bot pushed a commit that referenced this pull request Sep 26, 2025
Removes Docs as CODEOWNERS from the repo. As part of a project to reduce courtesy reviews, the Docs Team is limiting CODEOWNERS entries to repos that single-source content into the docs site or where our review is critical. We’re still happy to review content if you reach out to us directly. 1c00fa8
Sign up for free to join this conversation on GitHub. Already have an account? Sign in to comment
Projects
None yet
Development

Successfully merging this pull request may close these issues.

4 participants